Propagate non-retryable error messages to client #5929
Conversation
mark-pictor-csec
requested review from
MrAlias,
XSAM,
dashpole,
pellared and
dmathieu
as code owners
dmathieu
added
the
blocked:CLA
Could you split that into another PR? You will need to sign the CLA. |
Can do.
Yes, I am waiting on someone within my company (corp CLA). |
|
|
mark-pictor-csec
force-pushed
the
propagate-client-errors
branch
2 times, most recently
from
5c6dc2a
to
4c276b1
Compare
There were inconsistencies in closing the response body. For traces, the Close happened in a defer statement and any error was logged. Logs and metrics were less rigorous. It appeared Close() wasn't always called, and when it was, errors were returned sometimes and ignored at other times. This applies the defer logic from traces to the other two and removes other Close() calls. This was part of PR #5929, and has been split out as requested: #5929 (comment)).
PR open-telemetry#5541 (and issue open-telemetry#5536) enhance error handling, returning body text as part of the error. However, this is only done for retryable errors; if non-retryable, error text still does not propagate to clients. This PR adds handling of non-retryable errors, ensuring any body text is part of the message returned to the user's code. There is no change to the circumstances under which errors are reported, just an enhancement of the content of such an error.
mark-pictor-csec
force-pushed
the
propagate-client-errors
branch
from
4c276b1
to
e3332de
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5929 +/- ##
=====================================
Coverage 84.6% 84.6%
=====================================
Files 272 272
Lines 22897 22883 -14
=====================================
- Hits 19390 19380 -10
+ Misses 3163 3158 -5
- Partials 344 345 +1
|
|
@dmathieu CLA signed and the PR split from this is merged. Shall I create a new PR or are we good to go with this one? |
We are good with this one. |
pellared
removed
the
blocked:CLA
PR #5541 (and issue #5536) enhance error handling, returning body text as part of the error. However, this is only done for retryable errors; if non-retryable, error text still does not propagate to clients.
This PR adds handling of non-retryable errors, ensuring any body text is part of the message returned to the user's code. There is no change to the circumstances under which errors are reported, just an enhancement of the content of such an error.